gtksizegroup: Add explicit braces to avoid dangling else
authorKalev Lember <kalevlember@gmail.com>
Tue, 2 Apr 2013 22:35:27 +0000 (00:35 +0200)
committerKalev Lember <kalevlember@gmail.com>
Tue, 7 May 2013 11:59:53 +0000 (13:59 +0200)
Otherwise the else after the for loop becomes attached to the wrong if.

gtk/gtksizegroup.c

index e31dfa84eb1755b30b53f228eeff280941a16989..f84ed47f48c89b44890feb3395f0088bc6690d3f 100644 (file)
@@ -667,15 +667,22 @@ size_group_start_element (GMarkupParseContext *context,
   GSListSubParserData *data = (GSListSubParserData*)user_data;
 
   if (strcmp (element_name, "widget") == 0)
-    for (i = 0; names[i]; i++)
-      if (strcmp (names[i], "name") == 0)
-       data->items = g_slist_prepend (data->items, g_strdup (values[i]));
+    {
+      for (i = 0; names[i]; i++)
+        {
+          if (strcmp (names[i], "name") == 0)
+            data->items = g_slist_prepend (data->items, g_strdup (values[i]));
+        }
+    }
   else if (strcmp (element_name, "widgets") == 0)
-    return;
+    {
+      return;
+    }
   else
-    g_warning ("Unsupported type tag for GtkSizeGroup: %s\n",
-              element_name);
-
+    {
+      g_warning ("Unsupported type tag for GtkSizeGroup: %s\n",
+                 element_name);
+    }
 }
 
 static const GMarkupParser size_group_parser =